Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AA-427: Remove all mentions of 'aggregation' possible #8

Conversation

forshtat
Copy link

When opening a pull request to submit a new EIP, please use the suggested template: https://github.com/ethereum/EIPs/blob/master/eip-template.md

We have a GitHub bot that automatically merges some PRs. It will merge yours immediately if certain criteria are met:

  • The PR edits only existing draft PRs.
  • The build passes.
  • Your GitHub username or email address is listed in the 'author' header of all affected PRs, inside .
  • If matching on email address, the email address is the one publicly listed on your GitHub profile.

ducthotran2010 and others added 25 commits August 15, 2024 13:08
Merged by EIP-Bot.
Merged by EIP-Bot.
Merged by EIP-Bot.
* Update ERC-3770: Fix broken links and formatting

* use relative links only, for website
Merged by EIP-Bot.
Merged by EIP-Bot.
ERCS/erc-4337.md Outdated Show resolved Hide resolved
ERCS/erc-4337.md Outdated Show resolved Hide resolved
ERCS/erc-4337.md Outdated Show resolved Hide resolved
ERCS/erc-4337.md Outdated
* The `data` field SHOULD contain a `paymaster` or `aggregator` value, depending on the failed entity
* **code: -32505** - transaction rejected because paymaster (or signature aggregator) stake or unstake-delay is too low
* The `data` field SHOULD contain a `paymaster` or `aggregator` value, depending on the failed entity
* **code: -32504** - transaction rejected because paymaster is throttled/banned

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

irrelevant, as it is removed to an external ERC

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed all mentions of error codes.

sullof and others added 4 commits September 5, 2024 14:42
The `polyfill.io` domain was taken over by a malicious actor, and now
injects code that redirects to unrelated websites.

References:

<https://sansec.io/research/polyfill-supply-chain-attack>
<https://blog.cloudflare.com/automatically-replacing-polyfill-io-links-with-cloudflares-mirror-for-a-safer-internet/>

Reported-By: Elliott Green <[email protected]>
Co-authored-by: Dror Tirosh <[email protected]>
Co-authored-by: Dror Tirosh <[email protected]>
```

The `AggregatorStakeInfo` structure is further defined in [ERC-XXXX](link).
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO add actual link

@forshtat
Copy link
Author

Closed in favour of ethereum#627

@forshtat forshtat closed this Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.